Fix bulk_assign tests to not depend on removed member mutation#2064
Merged
Conversation
test_bulk_assign_data_rows and test_bulk_assign_data_rows_with_allowed_statuses relied on the project_based_user fixture, which creates an org member via the addMembersToOrganization mutation. That mutation was removed from the API back in intelligence #11829 (a2bdf13f69, 2022-11-15), so the fixture has been erroring at setup ever since (masked by the chronically-red integration suite). There is no public-API replacement that yields an assignable member without email-invite acceptance (invite_user returns a pending Invite, not a User). bulk_assign_data_rows only needs a valid member uid, so assign to the current authenticated user (client.get_user()) instead. This keeps the live bulk_assign_data_rows feature under test rather than dropping coverage. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
mrobers1982
requested review from
carl-l-chua,
gmadaan-hue,
golchin-shahriar,
gramos-lb,
lb-mwetter,
stan-issa and
vedsirdeshmukh
July 10, 2026 20:02
itssujan
approved these changes
Jul 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
test_bulk_assign_data_rowsandtest_bulk_assign_data_rows_with_allowed_statusesdepended on theproject_based_userfixture, which provisions an org member via theaddMembersToOrganizationmutation. That mutation was removed from the API in intelligence #11829 (a2bdf13f69, 2022-11-15), so the fixture has been erroring at setup ever since — invisible because the integration suite has been chronically red.Fix
bulk_assign_data_rowsis a live SDK feature; only its test setup was broken. There's no public-API way to create an assignable member without email-invite acceptance (organization.invite_user()returns a pendingInvite, not aUserwith a usableuid). Since the test only needs a valid memberuid, it now assigns to the current authenticated user viaclient.get_user(). This keeps the feature under test instead of deleting coverage.client.get_user()and dropped theproject_based_userdependency.test_bulk_assign_empty_listwas already independent and unchanged.Note
project_based_userremains (used only bytest_user_management::test_member_management, which is already@skipped). Its broader cleanup / member-management rewrite is out of scope here.Verification
ruff format --check+ruff checkclean (ruff 0.8.2, matching CI).🤖 Generated with Claude Code
Note
Low Risk
Test-only change with no production SDK or API surface modifications.
Overview
Integration tests for
bulk_assign_data_rowsno longer use theproject_based_userfixture, which relied on the removedaddMembersToOrganizationGraphQL mutation and failed at setup.Both affected tests now take the
clientfixture and useclient.get_user()as the assignee so they still exercise bulk assign with a valid memberuid.test_bulk_assign_empty_listis unchanged.Reviewed by Cursor Bugbot for commit b21eba0. Bugbot is set up for automated code reviews on this repo. Configure here.